154 feature request option to dry run #416
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ry-run # Conflicts: # lib/install.go # lib/symlink.go # main.go
MatrixCrawler
requested review from
warrensbox,
yermulnik,
jukie,
crablab and
MatthewJohn
as code owners
|
Actually, this feature obsoletes the --show-latest-pre and --show-latest-stable flags as you can you use |
yermulnik
approved these changes
Apr 26, 2024
|
@MatrixCrawler Apologies I haven't shouted earlier: > git status
On branch master
Your branch is up-to-date with 'origin/master'.
nothing to commit, working tree clean
> go run ./main.go --dry-run -u
# no output
> echo $?
0
> go run ./main.go -u
19:37:52.830 INFO Installing terraform at "/home/giermulnik/bin"
19:37:52.875 INFO Switched terraform to version "1.8.2"Seems like nothing happens with |
|
Also this but it's probably from something else: > go run ./main.go --latest-pre 1.7 --dry-run
19:42:43.031 ERROR Error parsing constraint "1.7": <nil>
19:42:43.031 INFO Invalid minor terraform version format.
Format should be #.# where # are numbers. For example, 0.11 is valid version
> go run ./main.go --latest-pre 1.7
19:41:54.254 INFO Installing terraform at "/home/giermulnik/bin"
19:41:54.336 INFO Downloading "https://releases.hashicorp.com/terraform/1.7.0-rc2/terraform_1.7.0-rc2_linux_amd64.zip"
19:41:54.336 INFO Downloading to "/home/giermulnik/.terraform.versions/terraform_1.7.0-rc2_linux_amd64.zip"
19:41:56.620 INFO 25939409 bytes downloaded
19:41:56.620 INFO Downloading "https://releases.hashicorp.com/terraform/1.7.0-rc2/terraform_1.7.0-rc2_SHA256SUMS"
19:41:56.620 INFO Downloading to "/home/giermulnik/.terraform.versions/terraform_1.7.0-rc2_SHA256SUMS"
19:41:57.343 INFO 1434 bytes downloaded
19:41:57.343 INFO Downloading "https://releases.hashicorp.com/terraform/1.7.0-rc2/terraform_1.7.0-rc2_SHA256SUMS.72D7468F.sig"
19:41:57.343 INFO Downloading to "/home/giermulnik/.terraform.versions/terraform_1.7.0-rc2_SHA256SUMS.72D7468F.sig"
19:41:57.420 INFO 566 bytes downloaded
19:41:57.420 INFO Verifying signature of checksum file...
19:41:57.427 INFO Checksum file signature verification successful.
19:41:57.485 INFO Deleting "/home/giermulnik/.terraform.versions/terraform_1.7.0-rc2_SHA256SUMS"
19:41:57.485 INFO Deleting "/home/giermulnik/.terraform.versions/terraform_1.7.0-rc2_SHA256SUMS.72D7468F.sig"
19:41:57.901 INFO Switched terraform to version "1.7.0-rc2"
19:41:57.901 ERROR Error parsing constraint "1.7": <nil>
19:41:57.901 INFO Invalid minor terraform version format.
Format should be #.# where # are numbers. For example, 0.11 is valid versionThe errors looks odd 😕 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added --dry-run flag
This will fix #154